Skip to content

Embed LICENSE.txt in NuGet package#13

Merged
DecSmith42 merged 4 commits into
mainfrom
copilot/embed-license-in-nuget
Jul 9, 2026
Merged

Embed LICENSE.txt in NuGet package#13
DecSmith42 merged 4 commits into
mainfrom
copilot/embed-license-in-nuget

Conversation

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

The repo's LICENSE.txt was not being packed into the output NuGet package, leaving consumers without license metadata or file.

Changes

  • Directory.Build.props: Added PackageLicenseFile pointing to LICENSE.txt, and included the file in the pack ItemGroup, mirroring the existing pattern used for icon.png and readme.md.
<PackageLicenseFile>LICENSE.txt</PackageLicenseFile>
...
<None Include="../../LICENSE.txt" Pack="true" PackagePath="\" />

Verified via dotnet pack that the resulting .nuspec now declares <license type="file">LICENSE.txt</license> and that LICENSE.txt is present in the .nupkg contents alongside the icon and readme.

Copilot AI requested review from Copilot and removed request for Copilot July 9, 2026 11:55
Copilot AI linked an issue Jul 9, 2026 that may be closed by this pull request
Copilot AI requested review from Copilot and removed request for Copilot July 9, 2026 11:58
Copilot AI changed the title [WIP] Embed license in NuGet package output Embed LICENSE.txt in NuGet package Jul 9, 2026
Copilot AI requested a review from DecSmith42 July 9, 2026 11:58
@DecSmith42 DecSmith42 marked this pull request as ready for review July 9, 2026 12:00
Copilot AI review requested due to automatic review settings July 9, 2026 12:00
@DecSmith42 DecSmith42 enabled auto-merge July 9, 2026 12:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the repo-wide NuGet packing configuration to ensure the project’s license text file is embedded in produced .nupkg artifacts, improving compliance and metadata completeness for consumers.

Changes:

  • Adds PackageLicenseFile metadata pointing to LICENSE.txt.
  • Packs LICENSE.txt into the NuGet package root alongside other package assets.

Comment thread Directory.Build.props
Comment thread Directory.Build.props Outdated
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Copilot AI review requested due to automatic review settings July 9, 2026 12:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@DecSmith42 DecSmith42 disabled auto-merge July 9, 2026 12:08
- Updated file inclusion paths in `Directory.Build.props` and `_atom.csproj` to use `$(MSBuildThisFileDirectory)` for consistency.
- Added `.gitattributes` to enforce LF endings on `.verified.txt` and `.received.txt` files.
- Updated copyright information in `LICENSE.txt` to reflect the correct author.

No functional changes introduced.
Copilot AI review requested due to automatic review settings July 9, 2026 12:46
@DecSmith42 DecSmith42 enabled auto-merge July 9, 2026 12:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread LICENSE.txt
@DecSmith42 DecSmith42 merged commit 66f0669 into main Jul 9, 2026
10 checks passed
@DecSmith42 DecSmith42 deleted the copilot/embed-license-in-nuget branch July 9, 2026 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Embed license in nuget

3 participants